Skip to content

Conversation

@leolost2605
Copy link
Member

@leolost2605 leolost2605 commented Dec 21, 2024

Instead of presenting the consumers of gestures with all the details about the gesture we just give them the action that should be triggered, and whether we should go forward (e.g. switch to the left with workspace switches or open the multitasking view) or backward (the opposite).

This helps

  1. with memory since we have to store less values (negligible but nice to have ig :)))
  2. with performance because we don't have to call GestureSettings.get_action twice in e.g. WindowManager
  3. with lines of code
  4. with preparation for more deduplication between one to one gestures and e.g. keyboard "gestures"

So win, win, win, win I guess? Let me know what you think :)

@leolost2605 leolost2605 requested a review from a team December 21, 2024 16:23
@leolost2605 leolost2605 mentioned this pull request Dec 21, 2024
18 tasks
@lenemter lenemter added the Conflicts Has conflicts with the target branch label Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Conflicts Has conflicts with the target branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants